Repository navigation
Conversation
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Upstream now unifies credentials under IFLY_* with XFEI_*/XFYUN_* fallback and fixes the speed-transcription digest, multipart chunking and task-query flow, so take those files as-is and drop the local transcription patches and their tests. Keep the image OCR, proofreading and PDF OCR packaging fixes, and expose the speed-transcription task options that SKILL.md documents but the CLI rejected (also submitted upstream as iflytek/iFly-Skills#106). Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
|
Refreshed this branch: merged current |
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Use GMT dates independent of process locale for image OCR and Hyper TTS, and lowercase OCR image suffixes. Cover both signing paths and image encoding cases with offline regressions. Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.
Reviewed by Cursor Bugbot for commit 5636e26. Configure here.
FenjuFu
changed the title
|
Merged current |
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
|
Merged current main (99b609a) in 10e2b3e to resolve the marketplace and README conflicts. The resolution keeps all upstream entries and the existing iFLYTEK entry; the plugin subtree is unchanged. No history was rewritten. The plugin validator, seven runtime regression tests, two Hyper TTS tests, Python compilation and diff checks against current main pass locally. All 13 commits in the PR range have matching sign-offs. The tests use mocked responses and make no live API calls. The broader pstack harness generator fails on Windows at generate-pstack-harness-trees.mjs:133: native relative paths use backslashes while the lookup uses forward slashes. The generator and its pstack/harness inputs are identical to current main. The pstack script suite subsequently finished with failures in the Windows/mise setup cases; it is not reported as passing. The new-head Cursor Bugbot check passed, while Validate plugins run 40372135382 requires maintainer approval. There is no independent DCO check on this repository. |
Summary
iflytek-skills, a third-party skill-only plugin with eight official iFLYTEK speech, OCR, transcription, proofreading, translation, and multimodal workflowsCloses #292.
Scope and provenance
The skill packages are based on
iflytek/iFly-Skills@beb3ae3a16d9ba952303acac1eb801f234d9c348, and the upstream Apache-2.0 license is included unchanged. Files modified for marketplace packaging carry a prominent notice.Included skills:
iflytek-hyper-ttsiflytek-image-understandingiflytek-ocr-invoiceiflytek-pdf-image-ocriflytek-speed-transcriptioniflytek-text-proofreadiflytek-translateiflytek-video-translateThe plugin has no hooks, background processes, MCP server, automatic dependency installation, or bundled credentials. The selected scripts use HTTPS or WSS service endpoints.
As proposed in #292, this initial version excludes the incomplete contract-review clients, the voice-cloning training flow (it still uses plain HTTP at the pinned revision), and the unrelated animated-diagram skill.
Packaging fixes
Upstream has since unified credentials under
IFLY_*(withXFEI_*/XFYUN_*fallback) and fixed the transcription request digest, multipart chunking, and task-query flow. Those parts of this PR now come straight from upstream, so the remaining packaging changes are:Hostheader with its signed endpoint--pdf-urland no dummy local path--output-type,--postproc-on,--enable-subtitle,--smoothproc,--colloqproc,--language-type,--dhw) that itsSKILL.mddocuments but the CLI rejected. This is also submitted upstream as fix(speed-transcription): expose documented task options in the CLI iflytek/iFly-Skills#106.Seven offline runtime regression tests and two Hyper TTS tests cover these cases.
Data boundary
Invoked skills send the requested text, image, PDF, audio, or video URL to the corresponding iFLYTEK API. The plugin README tells users to confirm organizational policy, service terms, entitlement, quota, and pricing before sending sensitive content.
Validation
node scripts/validate-plugins.mjspassespython -m compileall -q third_party/iflytek-skillspassesSigned-off-bytrailersNote
Medium Risk
New third-party integration that sends user content and credentials to external iFLYTEK APIs on skill invocation; marketplace core auth is unchanged but users must trust egress and env-based API keys.
Overview
Adds the iFLYTEK Skills third-party plugin to the Cursor marketplace: registers
iflytek-skillsinmarketplace.jsonand the root README, and shipsthird_party/iflytek-skills/with manifest, license, changelog, and eight agent skills (hyper TTS, image understanding, invoice OCR, PDF/image OCR, speed transcription, text proofread, translate, video translate).Each skill is invoked via local Python CLIs that call iFLYTEK HTTPS/WSS APIs using user-supplied
IFLY_*credentials (with legacy prefix fallback where documented). The bundle is based on upstreamiflytek/iFly-Skillswith marketplace-specific fixes: locale-independent auth dates, corrected image OCR HMAC host/signing, proofreadingHostheader alignment, URL-only PDF OCR, and extra speed-transcription CLI flags matchingSKILL.md. Regression/unit tests cover those packaging changes.The plugin README documents dependencies, credential names, data egress, and excluded upstream workflows (contract review, voice cloning, animated diagram). No MCP server, hooks, or bundled secrets.
Reviewed by Cursor Bugbot for commit 10e2b3e. Bugbot is set up for automated code reviews on this repo. Configure here.